Skip to content

chore: audit csql test instances#2117

Merged
kgala2 merged 9 commits intomainfrom
audit-csql-instances
Mar 7, 2025
Merged

chore: audit csql test instances#2117
kgala2 merged 9 commits intomainfrom
audit-csql-instances

Conversation

@kgala2
Copy link
Copy Markdown
Contributor

@kgala2 kgala2 commented Feb 20, 2025

Consolidate Cloud SQL instance usage for end-to-end connectivity testing in the following cases:

  1. MySQL IAM connection
  2. Postgres IAM connection

@kgala2 kgala2 requested a review from a team as a code owner February 20, 2025 22:16
@kgala2 kgala2 changed the title chore(csql): audit csql test instances - draft chore: audit csql test instances Feb 24, 2025
Copy link
Copy Markdown
Collaborator

@jackwotherspoon jackwotherspoon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ✅

@jackwotherspoon jackwotherspoon self-requested a review February 25, 2025 14:41
MYSQL_USER:${{ vars.GOOGLE_CLOUD_PROJECT }}/MYSQL_USER
MYSQL_PASS:${{ vars.GOOGLE_CLOUD_PROJECT }}/MYSQL_PASS
MYSQL_DB:${{ vars.GOOGLE_CLOUD_PROJECT }}/MYSQL_DB
MYSQL_IAM_CONNECTION_NAME:${{ vars.GOOGLE_CLOUD_PROJECT }}/MYSQL_JAVA_IAM_CONNECTION_NAME
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This may be a breaking change... this GraalVM test only runs nightly and not on PRs. It seems to use MYSQL_JAVA_IAM_CONNECTION_NAME

Comment thread .github/workflows/tests.yml Outdated
@kgala2 kgala2 requested a review from jackwotherspoon March 7, 2025 00:23
@kgala2 kgala2 merged commit 31917a4 into main Mar 7, 2025
17 checks passed
@kgala2 kgala2 deleted the audit-csql-instances branch March 7, 2025 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants